Skip to content

QVAC-14302 feat[api]: export RuntimeStats interface in OCR addon index.d.ts#1612

Merged
RamazTs merged 4 commits into
tetherto:mainfrom
RamazTs:feat/QVAC-14302-export-runtimestats-ocr
Apr 17, 2026
Merged

QVAC-14302 feat[api]: export RuntimeStats interface in OCR addon index.d.ts#1612
RamazTs merged 4 commits into
tetherto:mainfrom
RamazTs:feat/QVAC-14302-export-runtimestats-ocr

Conversation

@RamazTs

@RamazTs RamazTs commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Export a new RuntimeStats interface in packages/ocr-onnx/index.d.ts with fields: totalTime, detectionTime, recognitionTime, textRegionsCount
  • Fields match the C++ backend output from addon/pipeline/Pipeline.cpp (lines 478-483)
  • Existing OCRStats interface is preserved for backward compatibility

Files changed

  • packages/ocr-onnx/index.d.ts - added RuntimeStats interface (7 lines)

Verification

  • Confirmed field names match C++ backend (Pipeline.cpp)
  • Verified .d.ts syntax is well-formed (consistent indentation, semicolons, export keyword)
  • No build needed - typings-only change

Test plan

  • Verify SDK can import RuntimeStats from @qvac/ocr-onnx
  • Confirm no type conflicts with existing OCRStats

Asana: QVAC-14302

Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Resolve changelog/version conflict: bump version to 0.4.3 (0.4.1 and 0.4.2 already taken on main).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@RamazTs

RamazTs commented Apr 17, 2026

Copy link
Copy Markdown
Contributor Author

/review

@github-actions

Copy link
Copy Markdown
Contributor

Tier-based Approval Status

**PR Tier:** TIER1

**Current Status:** ✅ APPROVED

**Requirements:**
- 1 Team Member approval ✅ (2/1)
- 1 Team Lead OR Management approval ✅ (1/1)



---
*This comment is automatically updated when reviews change.*

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants